-
Notifications
You must be signed in to change notification settings - Fork 16.5k
Do not allow semicolon in partition_clause #48098
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do not allow semicolon in partition_clause #48098
Conversation
|
FYI this pull request was mentioned as associated with CVE-2025-30473 at https://www.openwall.com/lists/oss-security/2025/04/04/2 . Please consider adding "CVE-2025-30473" to the pull request title 🙏 |
What for? |
|
Visibility, transparency. |
Well. Changing title of PR after it's merged is wrong and it expects an extra effort on maintainers who do it in their free time. And we cannot refer to CVE before we announce it, and when you look at CVE announcement you have link to the PR. I think it's good enough. If you think you might want to increase visibility, I suggest you start your own project where you keep the links between CVE and PRs even more transparency. That would be a great relief for maintainers who do a lot of the work in their free time to keep your usage of the open-source projects they maintain secure. Would you like to start such a project? That would be greatly appreciated by all the open source maintainers who make their software available for free - mostly in their free time away from their families and other day job, Can we (open-source community) count on your help there @hartwork ? |
|
I think the whole community will thank you for that - here is your chance to actually contribute @hartwork |
|
@potiuk all my work on Open Source including libexpat is unpaid voluntary work including the security stuff and I do keep CVEs in pull request titles there if you would like to check: https://github.com/libexpat/libexpat/pulls?q=is%3Apr+is%3Aclosed+CVE . Does that change the picture for you? |
No. You are free to do what you want and spend as much time as you want on it. It's your free tiime to spend and it's you to decide what to do with it. Similarly, we decided what to do with our time, |
|
Hi, I want to reproduce the issue but I'm not that familier with airflow and couldn't find where |
Not allowing semicolon in parameterised fields.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in newsfragments.